Skip to content

[v10] Reduce DefaultIdleTimeout to 30s#23948

Merged
rosstimothy merged 1 commit intobranch/v10from
bot/backport-23943-branch/v10
Apr 1, 2023
Merged

[v10] Reduce DefaultIdleTimeout to 30s#23948
rosstimothy merged 1 commit intobranch/v10from
bot/backport-23943-branch/v10

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #23943 to branch/v10

An idle timeout of 6m keeps web api connections open too long on
Proxies that have any agents which do not have the changes from
#22832. Because the http.Server.IdleTimeout is set on all release
versions now, even when built with go.19, we experience this issue
because we no longer exercise the bug that was fixed in go1.20.
@rosstimothy rosstimothy enabled auto-merge March 31, 2023 23:43
@rosstimothy rosstimothy added this pull request to the merge queue Apr 1, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2023
@rosstimothy rosstimothy added this pull request to the merge queue Apr 1, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2023
@r0mant r0mant added this pull request to the merge queue Apr 1, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2023
@r0mant r0mant added this pull request to the merge queue Apr 1, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2023
@rosstimothy rosstimothy added this pull request to the merge queue Apr 1, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 1, 2023
@rosstimothy rosstimothy added this pull request to the merge queue Apr 1, 2023
Merged via the queue into branch/v10 with commit 1f8d304 Apr 1, 2023
@rosstimothy rosstimothy deleted the bot/backport-23943-branch/v10 branch April 1, 2023 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants